Skip to content
This repository was archived by the owner on Jan 11, 2023. It is now read-only.

[rake arm:release] Rescue error while publishing#972

Merged
vishrutshah merged 1 commit intoAzure:masterfrom
vishrutshah:rescue-publish
Sep 12, 2017
Merged

[rake arm:release] Rescue error while publishing#972
vishrutshah merged 1 commit intoAzure:masterfrom
vishrutshah:rescue-publish

Conversation

@vishrutshah
Copy link
Copy Markdown
Contributor

Swallowing errors while publishing to allow publishing of the only new version of the gems

Copy link
Copy Markdown
Contributor

@veronicagg veronicagg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approving as a temporary fix for recent publish transient error.

# Using execute method so that keys are not logged onto console
execute("gem push ./pkg/#{dir}-#{version}.gem" + (args[:key].nil? ? '' : " -k #{args[:key]}"))
rescue Exception => ex
puts "Error occurred while publishing #{dir}: #{ex}"
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would do caps on "ERROR"

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

that's true but i'll revert this changes once we release :)

@vishrutshah
Copy link
Copy Markdown
Contributor Author

Thanks for the quick review!!

@vishrutshah vishrutshah merged commit 59eeea5 into Azure:master Sep 12, 2017
@vishrutshah vishrutshah deleted the rescue-publish branch September 12, 2017 01:03
vishrutshah added a commit that referenced this pull request Sep 12, 2017
vishrutshah added a commit to vishrutshah/azure-sdk-for-ruby that referenced this pull request Sep 12, 2017
vishrutshah added a commit that referenced this pull request Sep 12, 2017
vishrutshah added a commit to vishrutshah/azure-sdk-for-ruby that referenced this pull request Sep 27, 2017
vishrutshah added a commit that referenced this pull request Sep 27, 2017
vishrutshah added a commit that referenced this pull request Sep 28, 2017
vishrutshah added a commit that referenced this pull request Sep 28, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants